Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add theme: bigger browser padding #851

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add theme: bigger browser padding #851

wants to merge 1 commit into from

Conversation

mr-cheff
Copy link
Member

@mr-cheff mr-cheff commented Dec 6, 2024

Add theme: bigger browser padding

This PR adds a new theme to the theme library.

Theme Details

@mr-cheff mr-cheff added the staged Theme still needs some changes label Dec 6, 2024
@froggy-jpg
Copy link

made some changes to the code

/* normal mode */
@media not (-moz-bool-pref: "zen.view.compact") {
    #tabbrowser-tabpanels {
        padding: 1px 16px 16px 1px !important;
    }

    @media (-moz-bool-pref: "zen.view.sidebar-expanded") {
        #TabsToolbar {
            margin: 0px 12px 6px 12px !important;
        }

    }

    @media not (-moz-bool-pref: "zen.view.sidebar-expanded") {
        #TabsToolbar {
            margin: 0px 0px 10px 0px !important;
        }
    }

    #tabbrowser-tabpanels:has(> [zen-split="true"]) {
        margin: 0px 10px 10px -6px !important;
    }
}

/* compact mode */
@media (-moz-bool-pref: "zen.view.compact") {
    #tabbrowser-tabpanels {
        padding: 16px !important;
    }

    /* hides toolbar*/
    @media (-moz-bool-pref: "zen.view.compact.hide-toolbar") {
        #tabbrowser-tabpanels {
            padding: 16px 16px 16px 1px !important;
        }

        @media not (-moz-bool-pref: "zen.view.sidebar-expanded") {
            #TabsToolbar {
                margin: 16px 0px 10px 0px !important;
            }
        }

        @media (-moz-bool-pref: "zen.view.sidebar-expanded") {
            #TabsToolbar {
                margin: 12px 12px 6px 12px !important;
            }
        }

        #tabbrowser-tabpanels:has(> [zen-split="true"]) {
            margin: 10px 10px 10px -6px !important;
        }
    }

    /* hides sidebar*/
    @media (-moz-bool-pref: "zen.view.compact.hide-tabbar") {
        #tabbrowser-tabpanels {
            padding: 16px !important;
        }

        #tabbrowser-tabpanels:has(> [zen-split="true"]) {
            margin: 0px 10px 10px 10px !important;

            @media (-moz-bool-pref: "zen.view.compact.hide-toolbar") {
                margin: 10px !important;
            }
        }
    }
}

/* splitview paddings */
#tabbrowser-tabbox {
    --zen-split-row-gap: 16px !important;
    --zen-split-column-gap: 16px !important;
}

/* remove padding in fullscreen */
@media (display-mode: fullscreen) {
    #tabbrowser-tabpanels {
        padding: 0px !important;
    }
}

/* remove weird scrollbar in compact mode */
* {
    scrollbar-width: none
}

@froggy-jpg
Copy link

also removed preferences because i couldnt get it working 100% the way i wanted

@mr-cheff
Copy link
Member Author

Why not just turn the preference up?

@froggy-jpg
Copy link

froggy-jpg commented Dec 11, 2024

do you ever feel stupid because all this time there was a way simpler solution than what you were doing initially?
anyway:

/* big mode */
@media (-moz-bool-pref: "uc.padding.big") {
    #main-window {
        --zen-element-separation: 12px !important;
      }
      
      #zen-sidebar-top-buttons {
        margin: 12px -6px 0px 6px!important;
      }
      
      #zen-sidebar-icons-wrapper {
        margin: 0px 6px !important;
        width: auto !important;
      }
      
      #navigator-toolbox {
        gap: 6px !important;
      }
          
      #tabbrowser-tabbox:has(#tabbrowser-tabpanels[zen-split-view="true"]) {
        --zen-split-row-gap: calc(var(--zen-element-separation) + 1px) !important;
        --zen-split-column-gap: calc(var(--zen-element-separation) + 18px) !important;
        margin-right: -12px !important;
      }  
}

/* bigger mode */
@media (-moz-bool-pref: "uc.padding.bigger") {
    #main-window {
        --zen-element-separation: 16px !important;
      }
      
      #zen-sidebar-top-buttons {
        margin: 16px -8px 0px 8px!important;
      }
      
      #zen-sidebar-icons-wrapper {
        margin: 0px 8px !important;
        width: auto !important;
      }
      
      #navigator-toolbox {
        gap: 8px !important;
      }
          
      #tabbrowser-tabbox:has(#tabbrowser-tabpanels[zen-split-view="true"]) {
        --zen-split-row-gap: calc(var(--zen-element-separation) + 1px) !important;
        --zen-split-column-gap: calc(var(--zen-element-separation) + 22px) !important;
        margin-right: -16px !important;
      }  
}

/* biggest mode */
@media (-moz-bool-pref: "uc.padding.biggest") {
    #main-window {
        --zen-element-separation: 20px !important;
      }
      
      #zen-sidebar-top-buttons {
        margin: 20px -10px 0px 10px!important;
      }
      
      #zen-sidebar-icons-wrapper {
        margin: 0px 10px !important;
        width: auto !important;
      }
      
      #navigator-toolbox {
        gap: 10px !important;
      }
          
      #tabbrowser-tabbox:has(#tabbrowser-tabpanels[zen-split-view="true"]) {
        --zen-split-row-gap: calc(var(--zen-element-separation) + 1px) !important;
        --zen-split-column-gap: calc(var(--zen-element-separation) + 26px) !important;
        margin-right: -20px !important;
      }   
}

/* compact sidebar fix */
@media (-moz-bool-pref: "zen.view.compact") {
  #zen-sidebar-top-buttons {
    margin: 0px !important;
  }

  #zen-sidebar-icons-wrapper {
    margin: 0px !important;
  }
}

also, preferences now also should work

@froggy-jpg
Copy link

no longer want this added, you can close the pr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
staged Theme still needs some changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants